Skip to content

refactor: integrate AppLayout into user profile page and simplify Adm…#1838

Merged
RUKAYAT-CODER merged 1 commit into
EarnQuestOne:mainfrom
akinboyewaSamson:feat/theme-toggle-all-routes
Jul 6, 2026
Merged

refactor: integrate AppLayout into user profile page and simplify Adm…#1838
RUKAYAT-CODER merged 1 commit into
EarnQuestOne:mainfrom
akinboyewaSamson:feat/theme-toggle-all-routes

Conversation

@akinboyewaSamson

@akinboyewaSamson akinboyewaSamson commented Jun 29, 2026

Copy link
Copy Markdown

closes #1673
closes #1721
closes #799
close #1697

Description

This PR addresses incomplete/broken layouts across administrative and profile routes by safely implementing the dark/light mode toggle in the AdminLayout and wiring up necessary context hooks to the user profile dashboard.

Changes Made

AdminLayout.tsx`:

  • Fixed broken layout file and complete structural tags.
  • Integrated the dynamic <ThemeToggle /> component into the top admin header.
  • Cleaned up the side navigation schema (Dashboard, Quests, Submissions, Users) to adapt seamlessly across zinc-50 light and zinc-900 dark modes.

Profile page.tsx:

  • Fixed incomplete file closure.
  • Correctly parsed the address dynamic slug param using Next.js useParams().
  • Connected user profile state variables (profileData, isLoading) and structured interaction callbacks (refetch, updateProfileData, follow, unfollow) down into the <UserProfile /> presentational component layout.

Checklist

  • Code compiles cleanly without shell syntax errors or dangling layout tokens.
  • Theme switching functions correctly inside administrative sub-routes.
  • State management logic maps perfectly down to the presentation tier of the profile view.
  • Branch history verified clean and ready for code review.

@drips-wave

drips-wave Bot commented Jun 29, 2026

Copy link
Copy Markdown

@akinboyewaSamson Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@RUKAYAT-CODER

Copy link
Copy Markdown
Contributor

Great job so far

There’s just one blockers — the workflow is failing. Could you take a look and fix it so all checks pass?
You could pull from the main to get the changes before pushing.

@RUKAYAT-CODER RUKAYAT-CODER merged commit 5513442 into EarnQuestOne:main Jul 6, 2026
1 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants